Skip to content

Fix multiple mini-nav dots highlighted when scrolling fast#615

Merged
martinyde merged 2 commits intofeature/timeline-conversion2from
fix/timeline-mini-nav-multiple-dots
Jan 27, 2026
Merged

Fix multiple mini-nav dots highlighted when scrolling fast#615
martinyde merged 2 commits intofeature/timeline-conversion2from
fix/timeline-mini-nav-multiple-dots

Conversation

@yepzdk
Copy link
Copy Markdown
Contributor

@yepzdk yepzdk commented Jan 27, 2026

Summary

  • Assign unique IDs to timeline notes (note-{paragraph_id}) instead of empty strings, so each note can be individually tracked
  • Improve scroll tracking to ignore hidden horizontal view duplicates by checking element dimensions before processing intersection events
  • Change scoring algorithm to prefer cards that contain the viewport center, ensuring small cards like notes are properly highlighted

Test plan

  • Navigate to a project page with a timeline containing notes
  • Scroll slowly through the timeline cards
  • Verify each card (including notes) highlights its corresponding mini-nav dot
  • Scroll quickly up and down
  • Verify only one dot is highlighted at a time

Fixes #609

🤖 Generated with Claude Code

- Assign unique IDs to timeline notes (note-{paragraph_id}) instead of
  empty strings, so each note can be individually tracked
- Improve scroll tracking to ignore hidden horizontal view duplicates
  by checking element dimensions before processing intersection events
- Change scoring algorithm to prefer cards that contain the viewport
  center, ensuring small cards like notes are properly highlighted

Fixes #609

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@yepzdk yepzdk requested a review from martinyde January 27, 2026 07:51
@martinyde martinyde merged commit 273d865 into feature/timeline-conversion2 Jan 27, 2026
15 of 21 checks passed
@martinyde martinyde deleted the fix/timeline-mini-nav-multiple-dots branch January 27, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants